-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Follow-up: XLSB Support #31215
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Follow-up: XLSB Support #31215
Conversation
@Rik-de-Kort yea your name should auto-generate as a contributor when we release Thanks! |
Co-authored-by: Rik-de-Kort <[email protected]>
Excellent, and you're welcome. Maybe next patch we go a bit quicker, eh? :P |
@WillAyd Happy to see the release is out, with my code :D However, my name does not appear to have been auto-generated in the contributors list. I do have the tag here though. Any ideas? |
Hmm @TomAugspurger any ideas on the contributor piece? Original PR of #29836 was part of 1.0 |
Updated min version for
pyxlsb
to 1.0.6 and removed the now unnecessary xfail for read from url. Turns out the other xfail is actually unnecessary so I've removed it.Question: is my name supposed to show up in the contributors list?
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff